Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Make autorestic configuration file path configurable #13

Merged
merged 3 commits into from
May 18, 2024
Merged

feat: Make autorestic configuration file path configurable #13

merged 3 commits into from
May 18, 2024

Conversation

micxer
Copy link
Contributor

@micxer micxer commented May 18, 2024

This commit updates the autorestic configuration file path to use the autorestic_config_file variable instead of hardcoding ~/.autorestic.yml. This change allows for greater flexibility and customization in specifying the location of the configuration file.

@dbrennand
Copy link
Owner

Hey @micxer 👋🏻

Thank you for your contribution to this role!

The changes look good to me. The lint and tests are failing, but looks completely unrelated to these changes. I'll take a look when I can.

@micxer
Copy link
Contributor Author

micxer commented May 18, 2024

Fixed it. Should be working now.

This commit updates the autorestic configuration file path to use the `autorestic_config_file` variable instead of hardcoding `~/.autorestic.yml`. This change allows for greater flexibility and customization in specifying the location of the configuration file.
This commit updates the GitHub Actions workflow file to use version v24 of ansible/ansible-lint instead of the archived version v6 ansible/ansible-lint-action. Also it update the action/checkout to v4.
@dbrennand dbrennand added the enhancement New feature or request label May 18, 2024
Copy link
Owner

@dbrennand dbrennand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thank you @micxer for your contribution! 🙂

@dbrennand dbrennand merged commit 0e20a3e into dbrennand:main May 18, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants